Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add updated Fedora image (respin) #537

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abitrolly
Copy link

Addresses #528.

Checksums are absent, and some C++ code needs to be written to fetch available respin version from https://dl.fedoraproject.org/pub/alt/live-respins/

I write it in Python, but not sure about my C++.

@grulja
Copy link
Collaborator

grulja commented Jul 11, 2022

I think this will just show the one particular version you added to the metadata, but we probably want this to be automated and always fetch latest respin, right? Also not sure what will happen if checksum and size are missing.

@abitrolly
Copy link
Author

@grulja yes. It also looks like files in repository are not used and https://getfedora.org/releases.json is fetched instead. Because file in this repo still lists F36 as beta, while 5.0.2 proposes to write 36 without this suffix.

Probably the best way is to add Live respin to the release scripts that update https://getfedora.org/releases.json

@grulja
Copy link
Collaborator

grulja commented Jul 12, 2022

@grulja yes. It also looks like files in repository are not used and https://getfedora.org/releases.json is fetched instead. Because file in this repo still lists F36 as beta, while 5.0.2 proposes to write 36 without this suffix.

It is used, but we also download latest releases.json file and use images from there if there are newer than what we ship.

Probably the best way is to add Live respin to the release scripts that update https://getfedora.org/releases.json

Yes, that would be the best solution.

@abitrolly
Copy link
Author

Added the request here https://pagure.io/fedora-web/websites/issue/257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants